Skip to content

docs: add placeholder for trail summary example screenshot#278

Merged
mbevc1 merged 3 commits into
mainfrom
mintlify/trail-summary-screenshot
Jun 18, 2026
Merged

docs: add placeholder for trail summary example screenshot#278
mbevc1 merged 3 commits into
mainfrom
mintlify/trail-summary-screenshot

Conversation

@mintlify

@mintlify mintlify Bot commented Jun 18, 2026

Copy link
Copy Markdown
Contributor

Adds a <Frame> slot in tutorials/trail_summaries_in_ci.md (under "What the markdown output looks like") for an example screenshot of kosli get trail … --output markdown rendered in a CI job summary.

The image at /images/tutorials/trail-summary-example.png is a 1×1 transparent placeholder — replace it with the actual screenshot before merging. A {/* TODO */} comment marks the spot.


Generated by Mintlify Agent.

Requested by: marko@kosli.com via Slack
Mintlify session: slack_1781798784.812579_C0AJTQ5633K

@mintlify mintlify Bot requested a review from a team as a code owner June 18, 2026 17:17
@claude

claude Bot commented Jun 18, 2026

Copy link
Copy Markdown
Contributor

Claude finished @mintlify[bot]'s task in 46s —— View job


Review

  • Read changed files and CLAUDE.md
  • Check navigation config for new pages
  • Review content quality and conventions
  • Post review findings

This PR looks good to merge. The earlier review suggestions (descriptive alt text, removing the {/* TODO */} comment) have been addressed in follow-up commits. Here's the current state:

tutorials/trail_summaries_in_ci.md

  • The <Frame> and image are correctly placed under "What the markdown output looks like".
  • Alt text is now descriptive for screen readers rather than duplicating the caption — nice.
  • No relative links, no missing nav entries (this is a modification to an existing page, not a new one).

images/tutorials/trail-summary-example.png

  • Despite the PR description saying "1×1 transparent placeholder", the image is actually a real 2560×1160 screenshot (262 KB) showing a trail summary with compliance status, attestation tables, and event history. Commit 975351d replaced the placeholder.

One minor note:

  • Suggestion (image size): The screenshot is 2560px wide. Mintlify will handle responsive scaling, but if load time matters you could optimize the PNG (e.g. pngquant or convert to WebP). Not blocking.

No critical or blocking issues found.

Comment thread tutorials/trail_summaries_in_ci.md Outdated
Comment thread tutorials/trail_summaries_in_ci.md
@mbevc1 mbevc1 enabled auto-merge (squash) June 18, 2026 17:27
@mbevc1 mbevc1 merged commit 17a41fd into main Jun 18, 2026
3 checks passed
@mbevc1 mbevc1 deleted the mintlify/trail-summary-screenshot branch June 18, 2026 17:27
@mintlify

mintlify Bot commented Jun 18, 2026

Copy link
Copy Markdown
Contributor Author

Preview deployment for your docs. Learn more about Mintlify Previews.

Project Status Preview Updated (UTC)
kosli 🔴 Failed Jun 18, 2026, 5:43 PM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant